- 
                Notifications
    You must be signed in to change notification settings 
- Fork 2k
add MiniMax model client #628
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
| Thanks, it has taken a long time for me to get to this but I am reviewing now. Thanks for such a comprehensive PR! | 
| @mxsl-gr I have sent you an email as I am not able to get an API key on the minimax web site. I enter my phone number to get a verification code, but the code never comes. | 
        
          
                models/spring-ai-minimax/src/main/java/org/springframework/ai/minimax/MiniMaxChatClient.java
          
            Show resolved
            Hide resolved
        
              
          
                models/spring-ai-minimax/src/main/java/org/springframework/ai/minimax/MiniMaxChatClient.java
          
            Show resolved
            Hide resolved
        
      | * | ||
| * <pre>{@code List.of("text1", "text2", "text3") or List.of(List.of(1, 2, 3), List.of(3, 4, 5))} </pre> | ||
| */ | ||
| public <T> ResponseEntity<EmbeddingList> embeddings(EmbeddingRequest embeddingRequest) { | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe the <T> needs to be removed.
        
          
                models/spring-ai-minimax/src/main/java/org/springframework/ai/minimax/api/MiniMaxApi.java
          
            Show resolved
            Hide resolved
        
              
          
                ...-ai-minimax/src/main/java/org/springframework/ai/minimax/api/common/MiniMaxApiException.java
          
            Show resolved
            Hide resolved
        
              
          
                spring-ai-docs/src/main/antora/modules/ROOT/pages/api/chat/minimax-chat.adoc
          
            Show resolved
            Hide resolved
        
      | I've merged the PR in 6b67401 Please review the comments above that were not address in the merge. Much thanks again | 
| 
 @markpollack Regarding the inability to receive verification codes from MiniMax or other model providers, it may be that they are only integrated with SMS providers in mainland China, and phone numbers from other countries or regions cannot receive the messages. In my email reply, I have provided the test API keys for MiniMax, Moonshot, and DeepSeek. | 
Hi, this PR is add
MiniMaxmodel client and has passed unit testing.I can provide my api_key if needed for testing
the PR content:
MiniMaxis a popular LLM in China Mainland recently, the link: MiniMaxFor some reasons, products from OpenAI and others can't be directly used in Chinese Mainland.
However ZhiPuAI, Moonshot, MiniMax, BaiChuanAI are referred to as the
Four Dragonsof China's LLM.If they can supported, it will further aid spring-ai to promotion in Chinese Mainland.
If necessary, I can take care of subsequent maintenance since I'm currently using them.
The
MoonshotPR is #596The
ZhiPuAIPR is #623I'll also provide PR for BaiChuan later.